fix #42: add support for web proxy#43
Conversation
|
@james-pplx - can we please get this merged; very useful to have a web proxy |
|
@kesku Any chance we could get this merged? Currently at work, I cannot use it due to the corporate proxy |
|
Hey @gurvancampion — unfortunately this PR is quite a bit out of date now. The MCP has gone through multiple updates since it was opened, and the web proxy implementation here conflicts with several newer changes. We’d need a fresh implementation based on the current codebase to move this forward. One other consideration is that we’re planning to migrate to an HTTP-based transport layer soon, which may break or complicate the current STDIO-based setup — so adding a proxy right now could introduce additional issues into the codebase. Regardless, I'll look into getting something like this merged soon. |
Thanks for the quick answer. Let me know if I can help. I could push a new MR based on the existing codebase, or wait till you have migrated to re-check at that time |
|
I'll look at it. |
|
Just an update here I’ve implemented proxy support in #66, which is now merged into main. This is a temporary solution since as stated previously we'll be migrating to streamable HTTP transport soon. Shouldn’t affect end users in the meantime! Thanks again @ojacques and @gurvancampion for flagging this! |
No description provided.